-
-
Notifications
You must be signed in to change notification settings - Fork 814
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
core#1346: Tagsets should display vertically in activity/new case forms #15649
Conversation
(Standard links)
|
@demeritcowboy can you review this? |
Yep I can review. |
Thanks this looks like a good improvement for tags but since some unrelated fields are being moved around the screen I have a question below.
|
@monishdeb @demeritcowboy I think we should move duration under Case Start Date since that is consistent with how we show on Activity screen as well. Again for tagsets for Case they are displayed at the very bottom and for Activity they are above Attachment, Repeat and Follow tabs. Whatever we decide they should be consistent across screens Thoughts? |
Could moving fields be a separate lab ticket since there's probably 20 different opinions on how this screen should look? This seemed like it was just about making multiple tagsets stack vertically instead of horizontally on activities, and in fact they were already vertical on open case just left-aligned with the left border. So it doesn't seem like any other fields need to be moved to achieve stacking. And I'm noticing this has changed the intent of the tableLayout parameter and now it's no longer possible to call the common tagset template to stack horizontally. Maybe that's ok just now tableLayout = true means stack vertically and align closer to the center, and tableLayout = false means stack vertically and align to the left. The only other place in core that tableLayout = true seems to be attachment tags, so they also now stack vertically too, but that seems fine. |
@demeritcowboy functionality to use tableLayout = true is still intact. Its just mean we are using tabular format to show tagset, so I have used condition to append the table tag only if tableLayout = true. Earlier the horizontal arrangement of tagset seems broken because we ae using div elements inside table. As per the patch we have clearly differentiated the outlook based on tableLayout parameter. As per UI design, I tried it to be consistent with the other forms like @yashodha @demeritcowboy will it be ok if I revert the Case tpl changes so not to tweak with the design pattern? Will it be ok to merge then? |
Yes thanks if the line 93 block is put back I think it's good to go. FYI you might be interested in this which I will put in the extension directory later today: https://gitlab.com/physician-health-program-of-bc/opencaserearranger |
@monishdeb sounds good :) |
@demeritcowboy @yashodha updated the PR |
@monishdeb thanks, will check |
Hi @yashodha was there something specific you were looking to check since otherwise this one seems good to go? |
@demeritcowboy looks good, merging this. |
Overview
Currently when there are more than one tagsets configured for activity and/or case, on the new activity or case form the tagset are listed horizontally. Please check the before/after screenshots for the style fixes:
Before
New Activity form:
New Case form:
After
New Activity form:
New Case form:
Comments
ping @lcdservices @colemanw @eileenmcnaughton